-
Notifications
You must be signed in to change notification settings - Fork 614
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
support for a custom apt source release #1561
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
IMHO we shouldn't do this and instead drop 18.04. The standard support of 18.04 has ended and clearly PostgreSQL no longer wants to supports it since they dropped it from their repositories (on 2023-11-05 according to https://wiki.postgresql.org/wiki/Apt#News).
If anything, we can introduce a parameter to apt_postgresql_org
for the release so you can override it via Hiera. In fact, I think that'd be wise for everything including the GPG keys.
bcb9dbb
to
503bee0
Compare
Useful in cases where postgresql has archived support for the platform.
503bee0
to
29ef7ff
Compare
Reworked. Thanks for the constructive feedback. I think the overall design pattern could use attention which is why I'm not touching key management yet. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can't properly review this right now, but at first glance this looks like it addresses my concerns.
postgresql::repo::baseurl: https://apt-archive.postgresql.org/pub/repos/apt/ | ||
postgresql::repo::release: "%{facts.os.distro.codename}-pgdg-archive" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Having archive repos is much cleaner than using a different distro. I much prefer this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the overall design pattern could use attention which is why I'm not touching key management yet.
I think #1563 would be a major step in that direction. If you just need to provide a URL without an ID then it's way easier.
Postgresql no longer specifically supports Ubuntu 18.04, but still supports Debian 10 which can be used until Puppet drops support for Ubuntu 18.04.
Summary
Postgresql recommends using the Debian buster repo for Ubuntu bionic.
Additional Context
Temporary until Puppet no longer supports Ubuntu 18.04.
Related Issues (if any)
voxpupuli/puppet-puppetboard#402
Checklist